-
Notifications
You must be signed in to change notification settings - Fork 7
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
Make describe method work on other people's tables #15
base: master
Are you sure you want to change the base?
Conversation
Hi Joe, thanks for the patch. I have a couple of comments
|
Hi Erin, Definitely good point on the second one - I didn’t actually know there were global tables! For example I can list the contents of coadd_objects but can’t see it on any kind of list of tables I can generate. Any ideas? I think this may be related to your first point, too - there may be tables I don’t know exist to test it on. I’ll change it to leave it without a username by default, definitely. Joe On 30 Oct 2014, at 14:04, Erin Sheldon [email protected] wrote:
|
by global table I just mean it is visible in the global namespace. You are right that it does have an owner, but has just been made more visible. |
Okay. So who owns the table (or whatever it is) called COADD_OBJECTS? I can’t see it listed anywhere. Joe On 30 Oct 2014, at 14:22, Erin Sheldon [email protected] wrote:
|
I have no idea! |
I've concluded I don't really understand Oracle on any level. Better leave this PR alone for now. |
I agree about oracle. I find it very opaque and the documentation hard to find, follow. It might be a good idea to just ask Todd how he would do it. |
Yeah. I've decided to avoid asking the DM people too many questions until after the Y1 release, though. This can wait until then!. |
Fixes issue #14 which I opened! With this you can do, e.g.:
import desdb
conn=desdb.Connection()
conn.describle("joezuntz.im3shape_v7_r")
and it will work as expected. The old mode will still work too.